Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

This commit allows you to disable logging for .sync() methods on Sequeli... #937

Merged
merged 5 commits into from Nov 3, 2013

Conversation

durango
Copy link
Member

@durango durango commented Sep 24, 2013

...ze and DAOFactories. This closes #763

@sdepold
Copy link
Member

sdepold commented Sep 25, 2013

hmm failed for pg?

@sdepold
Copy link
Member

sdepold commented Oct 1, 2013

@durango still failing :)

@durango
Copy link
Member Author

durango commented Oct 1, 2013

yeah ill go through my list of PRs tonight :)

@sdepold
Copy link
Member

sdepold commented Oct 1, 2013

hm as IRC seems to have some serious problems atm, I just wanted to make sure that I let u know that i will take care of the broken test tomorrow.

@sdepold
Copy link
Member

sdepold commented Oct 15, 2013

pg is broken.

@durango
Copy link
Member Author

durango commented Oct 29, 2013

All good @sdepold :)

janmeier added a commit that referenced this pull request Nov 3, 2013
This commit allows you to disable logging for .sync() methods on Sequeli...
@janmeier janmeier merged commit 3c29af3 into sequelize:master Nov 3, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add option {logging: false} to sync() method
3 participants